styleproperty: Remove unuse args from register()
authorBenjamin Otte <otte@redhat.com>
Sat, 31 Dec 2011 14:50:56 +0000 (15:50 +0100)
committerBenjamin Otte <otte@redhat.com>
Mon, 9 Jan 2012 17:37:52 +0000 (18:37 +0100)
Those arguments are only relevant for shorthands.

gtk/gtkstyleproperties.c
gtk/gtkstyleproperty.c
gtk/gtkstylepropertyprivate.h

index fc47984b6bf4d4b8ac30e9238710782df8ca4393..b3ef81b3cc69d7be8a4d5efc6378a6d23e0ad9e8 100644 (file)
@@ -377,9 +377,6 @@ gtk_style_properties_register_property (GtkStylePropertyParser  parse_func,
                                 parse_func,
                                 NULL,
                                 NULL,
-                                NULL,
-                                NULL,
-                                NULL,
                                 NULL);
 }
 
index d61fcec448ec7c187c1422bc67685c01d1c68b86..71d957532bf9bc5cf9b550ffc026cd000a1f6f4b 100644 (file)
@@ -2095,10 +2095,7 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   rgba_init (&rgba, 0, 0, 0, 0);
   g_value_set_boxed (&value, &rgba);
   _gtk_style_property_register           (g_param_spec_boxed ("background-color",
@@ -2107,12 +2104,9 @@ gtk_style_property_init_properties (void)
                                           GDK_TYPE_RGBA, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           transparent_color_value_parse,
                                           NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   g_value_unset (&value);
 
   g_value_init (&value, G_TYPE_STRV);
@@ -2123,12 +2117,9 @@ gtk_style_property_init_properties (void)
                                                               G_TYPE_STRV, 0),
                                           GTK_STYLE_PROPERTY_INHERIT,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           font_family_parse,
                                           font_family_value_print,
-                                          &value,
-                                          NULL);
+                                          &value);
   g_value_unset (&value);
   _gtk_style_property_register           (g_param_spec_enum ("font-style",
                                                              "Font style",
@@ -2139,9 +2130,6 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          NULL,
                                           NULL);
   _gtk_style_property_register           (g_param_spec_enum ("font-variant",
                                                              "Font variant",
@@ -2152,9 +2140,6 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          NULL,
                                           NULL);
   /* xxx: need to parse this properly, ie parse the numbers */
   _gtk_style_property_register           (g_param_spec_enum ("font-weight",
@@ -2166,9 +2151,6 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          NULL,
                                           NULL);
   g_value_init (&value, G_TYPE_DOUBLE);
   g_value_set_double (&value, 10);
@@ -2180,10 +2162,7 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   g_value_unset (&value);
 
   _gtk_style_property_register           (g_param_spec_boxed ("text-shadow",
@@ -2194,9 +2173,6 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          NULL,
                                           NULL);
 
   _gtk_style_property_register           (g_param_spec_boxed ("icon-shadow",
@@ -2207,9 +2183,6 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          NULL,
                                           NULL);
 
   gtk_style_properties_register_property (NULL,
@@ -2284,11 +2257,8 @@ gtk_style_property_init_properties (void)
                                                               GTK_TYPE_CSS_BORDER_CORNER_RADIUS, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           border_corner_radius_value_parse,
                                           border_corner_radius_value_print,
-                                          NULL,
                                           NULL);
   _gtk_style_property_register           (g_param_spec_boxed ("border-top-right-radius",
                                                               "Border top right radius",
@@ -2296,11 +2266,8 @@ gtk_style_property_init_properties (void)
                                                               GTK_TYPE_CSS_BORDER_CORNER_RADIUS, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           border_corner_radius_value_parse,
                                           border_corner_radius_value_print,
-                                          NULL,
                                           NULL);
   _gtk_style_property_register           (g_param_spec_boxed ("border-bottom-right-radius",
                                                               "Border bottom right radius",
@@ -2308,11 +2275,8 @@ gtk_style_property_init_properties (void)
                                                               GTK_TYPE_CSS_BORDER_CORNER_RADIUS, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           border_corner_radius_value_parse,
                                           border_corner_radius_value_print,
-                                          NULL,
                                           NULL);
   _gtk_style_property_register           (g_param_spec_boxed ("border-bottom-left-radius",
                                                               "Border bottom left radius",
@@ -2320,11 +2284,8 @@ gtk_style_property_init_properties (void)
                                                               GTK_TYPE_CSS_BORDER_CORNER_RADIUS, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           border_corner_radius_value_parse,
                                           border_corner_radius_value_print,
-                                          NULL,
                                           NULL);
 
   gtk_style_properties_register_property (NULL,
@@ -2353,48 +2314,36 @@ gtk_style_property_init_properties (void)
                                                               GDK_TYPE_RGBA, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           transparent_color_value_parse,
                                           NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   _gtk_style_property_register           (g_param_spec_boxed ("border-right-color",
                                                               "Border right color",
                                                               "Border right color",
                                                               GDK_TYPE_RGBA, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           transparent_color_value_parse,
                                           NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   _gtk_style_property_register           (g_param_spec_boxed ("border-bottom-color",
                                                               "Border bottom color",
                                                               "Border bottom color",
                                                               GDK_TYPE_RGBA, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           transparent_color_value_parse,
                                           NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   _gtk_style_property_register           (g_param_spec_boxed ("border-left-color",
                                                               "Border left color",
                                                               "Border left color",
                                                               GDK_TYPE_RGBA, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           transparent_color_value_parse,
                                           NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   g_value_unset (&value);
 
   gtk_style_properties_register_property (NULL,
@@ -2432,10 +2381,7 @@ gtk_style_property_init_properties (void)
                                           NULL,
                                           NULL,
                                           NULL,
-                                          NULL,
-                                          NULL,
-                                          &value,
-                                          NULL);
+                                          &value);
   g_value_unset (&value);
   gtk_style_properties_register_property (NULL,
                                           g_param_spec_object ("engine",
@@ -2455,11 +2401,8 @@ gtk_style_property_init_properties (void)
                                                               G_TYPE_PTR_ARRAY, 0),
                                           0,
                                           NULL,
-                                          NULL,
-                                          NULL,
                                           bindings_value_parse,
                                           bindings_value_print,
-                                          NULL,
                                           NULL);
 
   /* initialize shorthands last, they depend on the real properties existing */
@@ -2511,63 +2454,47 @@ void
 _gtk_style_property_register (GParamSpec               *pspec,
                               GtkStylePropertyFlags     flags,
                               GtkStylePropertyParser    property_parse_func,
-                              GtkStyleUnpackFunc        unpack_func,
-                              GtkStylePackFunc          pack_func,
                               GtkStyleParseFunc         parse_func,
                               GtkStylePrintFunc         print_func,
-                              const GValue *            initial_value,
-                              GtkStyleUnsetFunc         unset_func)
+                              const GValue *            initial_value)
 {
   GtkStyleProperty *node;
 
-  g_return_if_fail ((pack_func == NULL) == (unpack_func == NULL));
-
-  if (pack_func == NULL)
-    node = g_object_new (GTK_TYPE_CSS_STYLE_PROPERTY,
-                         "name", pspec->name,
-                         NULL);
-  else
-    node = g_object_new (GTK_TYPE_CSS_SHORTHAND_PROPERTY,
-                         "name", pspec->name,
-                         NULL);
+  node = g_object_new (GTK_TYPE_CSS_STYLE_PROPERTY,
+                       "name", pspec->name,
+                       NULL);
   node->flags = flags;
   node->pspec = pspec;
   node->property_parse_func = property_parse_func;
-  node->pack_func = pack_func;
-  node->unpack_func = unpack_func;
   node->parse_func = parse_func;
   node->print_func = print_func;
-  node->unset_func = unset_func;
 
-  if (!GTK_IS_CSS_SHORTHAND_PROPERTY (node))
-    {
-      _gtk_style_property_generate_id (node);
+  _gtk_style_property_generate_id (node);
 
-      /* initialize the initial value */
-      if (initial_value)
+  /* initialize the initial value */
+  if (initial_value)
+    {
+      g_value_init (&node->initial_value, G_VALUE_TYPE (initial_value));
+      g_value_copy (initial_value, &node->initial_value);
+    }
+  else
+    {
+      g_value_init (&node->initial_value, pspec->value_type);
+      if (pspec->value_type == GTK_TYPE_THEMING_ENGINE)
+        g_value_set_object (&node->initial_value, gtk_theming_engine_load (NULL));
+      else if (pspec->value_type == PANGO_TYPE_FONT_DESCRIPTION)
+        g_value_take_boxed (&node->initial_value, pango_font_description_from_string ("Sans 10"));
+      else if (pspec->value_type == GDK_TYPE_RGBA)
         {
-          g_value_init (&node->initial_value, G_VALUE_TYPE (initial_value));
-          g_value_copy (initial_value, &node->initial_value);
+          GdkRGBA color;
+          gdk_rgba_parse (&color, "pink");
+          g_value_set_boxed (&node->initial_value, &color);
         }
-      else
+      else if (pspec->value_type == GTK_TYPE_BORDER)
         {
-          g_value_init (&node->initial_value, pspec->value_type);
-          if (pspec->value_type == GTK_TYPE_THEMING_ENGINE)
-            g_value_set_object (&node->initial_value, gtk_theming_engine_load (NULL));
-          else if (pspec->value_type == PANGO_TYPE_FONT_DESCRIPTION)
-            g_value_take_boxed (&node->initial_value, pango_font_description_from_string ("Sans 10"));
-          else if (pspec->value_type == GDK_TYPE_RGBA)
-            {
-              GdkRGBA color;
-              gdk_rgba_parse (&color, "pink");
-              g_value_set_boxed (&node->initial_value, &color);
-            }
-          else if (pspec->value_type == GTK_TYPE_BORDER)
-            {
-              g_value_take_boxed (&node->initial_value, gtk_border_new ());
-            }
-          else
-            g_param_value_set_default (pspec, &node->initial_value);
+          g_value_take_boxed (&node->initial_value, gtk_border_new ());
         }
+      else
+        g_param_value_set_default (pspec, &node->initial_value);
     }
 }
index 5577b5fb95b1e73b32c8582a439607e5109337a5..8926ce1e3964e963b33f7f6f5d19a5474337f686 100644 (file)
@@ -91,12 +91,9 @@ const char *             _gtk_style_property_get_name      (GtkStyleProperty
 void                     _gtk_style_property_register      (GParamSpec             *pspec,
                                                             GtkStylePropertyFlags   flags,
                                                             GtkStylePropertyParser  property_parse_func,
-                                                            GtkStyleUnpackFunc      unpack_func,
-                                                            GtkStylePackFunc        pack_func,
                                                             GtkStyleParseFunc       parse_func,
                                                             GtkStylePrintFunc       print_func,
-                                                            const GValue           *initial_value,
-                                                            GtkStyleUnsetFunc       unset_func);
+                                                            const GValue           *initial_value);
 
 gboolean                 _gtk_style_property_is_inherit    (const GtkStyleProperty *property);
 guint                    _gtk_style_property_get_id        (const GtkStyleProperty *property);